Skip to content

release resources in PDAbstractContentStream#465

Closed
valerybokov wants to merge 1 commit into
apache:trunkfrom
valerybokov:release-resources-in-PDAbstractContentStream
Closed

release resources in PDAbstractContentStream#465
valerybokov wants to merge 1 commit into
apache:trunkfrom
valerybokov:release-resources-in-PDAbstractContentStream

Conversation

@valerybokov
Copy link
Copy Markdown

No description provided.

@msahyoun
Copy link
Copy Markdown

msahyoun commented Jun 2, 2026

    // Release large data structures immediately so the GC can reclaim memory
    // even if callers hold a reference to this COSDocument instance.
    
    Is this a copy and paste error? This is not COSDocument

@THausherr
Copy link
Copy Markdown
Contributor

I ran my rendering tests with a trace on these structures. Most of the time they're empty or have one element. Which isn't surprising considering that it's a stack. That there is 1 element is because it isn't required to include everything in q...Q operators.

@valerybokov valerybokov closed this Jun 5, 2026
@valerybokov valerybokov deleted the release-resources-in-PDAbstractContentStream branch June 5, 2026 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants